Skip to content

chore: fill or remove RSR template placeholders - #52

Merged
hyperpolymath merged 7 commits into
mainfrom
fix/rsr-placeholders
Jul 28, 2026
Merged

chore: fill or remove RSR template placeholders#52
hyperpolymath merged 7 commits into
mainfrom
fix/rsr-placeholders

Conversation

@hyperpolymath

@hyperpolymath hyperpolymath commented Jul 28, 2026

Copy link
Copy Markdown
Owner

openssf-compliance.yml fails when any of the thirteen files it checks still contains a {{PLACEHOLDER}} token. This clears them, with no invention.

  • Deleted the TEMPLATE INSTRUCTIONS (delete this block before publishing) comment — the template says to delete it, and it is where every legend line lived, so a large share of the reported tokens were the file documenting its own placeholders.
  • Filled what the repository itself supplies: owner and repo from the git remote, project name, year, forge, main branch, contact email.
  • Removed PGP and website lines. https://github.com/<user>.gpg returns HTTP 200 for every account; with no key uploaded the body is a stub reading "This user hasn't uploaded any GPG keys". No key is published for either account, and commit signing here is SSH — unrelated. The template sanctions this: "Optional: Remove sections that don't apply (e.g. PGP if you don't use it)."\n\nA security policy telling a researcher to encrypt to a key that does not exist is worse than one that does not mention encryption.\n\nWhere applicable, this also fixes a misrouted advisory URL. A previous just init copied already-filled templates out of squisher-corpus, so 51 repositories directed vulnerability reports to hyperpolymath/squisher-corpus — a repository unrelated to the code being reported, meaning the affected maintainer never saw them. Repointed from this repository's own git remote.

Summary by Gitar

  • Added configuration & governance:
    • Added mise.toml for tool versioning and environment management
    • Added GOVERNANCE.md, MAINTAINERS, and ARCHITECTURE.md project files
  • License updates:
    • Added full license text under LICENSES/AGPL-3.0-or-later.txt and updated other license files
  • GitHub integration:
    • Added funding.yml and updated workflow permissions across multiple GitHub action files

This will update automatically on new commits.

hyperpolymath and others added 6 commits July 25, 2026 09:45
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
openssf-compliance.yml fails when any of the thirteen files it checks
still contains a {{PLACEHOLDER}} token. This clears them.

Three kinds of change, no invention:

The "TEMPLATE INSTRUCTIONS (delete this block before publishing)" comment
is deleted. The template says to delete it, and it is where every legend
line lives -- so a large share of the reported tokens were the file
documenting its own placeholders, not real unfilled fields.

Tokens derivable from the repository are filled: owner and repo from the
git remote, project name, year, forge, main branch, contact email.

PGP and website lines are removed rather than filled, because nothing
true could go in them. https://github.com/<user>.gpg returns HTTP 200 for
every account; with no key uploaded the body is a stub reading "This user
hasnt uploaded any GPG keys". No key is published for either account
here, and commit signing in this estate is SSH, which is unrelated. Only
one repository in the estate has a domain, so {{WEBSITE}} likewise has no
correct value. The template sanctions this: "Optional: Remove sections
that dont apply (e.g. PGP if you dont use it)." A security policy telling
a researcher to encrypt to a key that does not exist is worse than one
that does not mention encryption.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
Comment thread mise.toml
Comment thread mise.toml
@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

CI failed: CI failed due to a missing GitHub action repository reference ('hyperpolymath/a2ml-validate-action') and a governance violation from banned ReScript files in the repository.

Overview

Two distinct failure patterns were found across 3 analyzed CI logs: a missing custom GitHub Actions dependency and a repository governance check failure due to banned language files (ReScript .res). Both failures are directly related to the repository state and changes.

Failures

Missing GitHub Action Repository (confidence: high)

  • Type: dependency
  • Affected jobs: 90326185239, 90326185308
  • Related to change: yes
  • Root cause: The workflow references a custom GitHub action 'hyperpolymath/a2ml-validate-action' which either does not exist, is private without proper access credentials, or has been renamed/deleted.
  • Suggested fix: Verify the repository name for 'hyperpolymath/a2ml-validate-action', ensure it is public or accessible to the workflow, or update the workflow file to point to the correct path/version of the action.

Banned Language File Violation (confidence: high)

  • Type: tooling
  • Affected jobs: 90326188471
  • Related to change: yes
  • Root cause: The repository contains banned language files (tests/SafeWhitespace_test.res) violating the cicd_rules/banned_language_file rule which requires ReScript files to be replaced by AffineScript.
  • Suggested fix: Remove or convert tests/SafeWhitespace_test.res to AffineScript, or add a # hypatia:ignore cicd_rules/banned_language_file pragma to the file if it is intentionally exempt.

Summary

  • Change-related failures: 2 failures (missing action dependency and governance policy violation for banned ReScript files)
  • Infrastructure/flaky failures: 0
  • Recommended action: Fix or remove the missing workflow action reference, and either convert/remove the banned ReScript file (tests/SafeWhitespace_test.res) or declare an appropriate hypatia exemption.
Code Review ✅ Approved 2 resolved / 2 findings

Adds estate-wide security compliance files, governance documents, and configuration templates, but mise.toml contains invalid syntax where task commands are placed under [alias] and uses an incorrect Deno tool name.

✅ 2 resolved
Bug: mise.toml task commands placed under [alias] won't run

📄 mise.toml:51-57
In mise, [alias] defines tool/version aliases, not runnable commands; tasks must be declared under [tasks.<name>] (e.g. [tasks.build] run = "..."). As written, build/test/lint/fmt/task will be treated as (invalid) tool aliases and mise run build etc. will not work, so the intended task-runner shortcuts silently do nothing. Move these into [tasks.*] tables with a run key.

Bug: mise.toml likely uses wrong Deno tool name 'denojs'

📄 mise.toml:10
The mise registry short name for Deno is deno, not denojs. As written, mise install may fail to resolve denojs = "latest" (or resolve to an unintended backend). Verify against the mise registry and rename to deno if confirmed.

Tip

Comment Gitar fix CI to trigger a fix.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@hyperpolymath
hyperpolymath merged commit 434924b into main Jul 28, 2026
16 of 20 checks passed
@hyperpolymath
hyperpolymath deleted the fix/rsr-placeholders branch July 28, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant